From 2bb48590316289729cb5b2372120c768de65af74 Mon Sep 17 00:00:00 2001 From: Matthias Clasen Date: Sat, 27 Mar 2021 16:23:40 -0400 Subject: [PATCH] ngl: Fix an uninitialized read We were using the wrong matrix here. valgrind pointed this point. --- gsk/ngl/gsknglrenderjob.c | 13 ++++++------- 1 file changed, 6 insertions(+), 7 deletions(-) diff --git a/gsk/ngl/gsknglrenderjob.c b/gsk/ngl/gsknglrenderjob.c index 8e4246d8b7..026f2be6e7 100644 --- a/gsk/ngl/gsknglrenderjob.c +++ b/gsk/ngl/gsknglrenderjob.c @@ -399,7 +399,6 @@ static void extract_matrix_metadata (GskNglRenderModelview *modelview) { float dummy; - graphene_matrix_t m; gsk_transform_to_matrix (modelview->transform, &modelview->matrix); @@ -428,14 +427,14 @@ extract_matrix_metadata (GskNglRenderModelview *modelview) /* TODO: 90% sure this is incorrect. But we should never hit this code * path anyway. */ graphene_vec3_init (&col1, - graphene_matrix_get_value (&m, 0, 0), - graphene_matrix_get_value (&m, 1, 0), - graphene_matrix_get_value (&m, 2, 0)); + graphene_matrix_get_value (&modelview->matrix, 0, 0), + graphene_matrix_get_value (&modelview->matrix, 1, 0), + graphene_matrix_get_value (&modelview->matrix, 2, 0)); graphene_vec3_init (&col2, - graphene_matrix_get_value (&m, 0, 1), - graphene_matrix_get_value (&m, 1, 1), - graphene_matrix_get_value (&m, 2, 1)); + graphene_matrix_get_value (&modelview->matrix, 0, 1), + graphene_matrix_get_value (&modelview->matrix, 1, 1), + graphene_matrix_get_value (&modelview->matrix, 2, 1)); modelview->scale_x = graphene_vec3_length (&col1); modelview->scale_y = graphene_vec3_length (&col2); -- 2.30.2